Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding support for Universal Binary for MacOS on M1 #450

Merged
merged 8 commits into from
Jan 23, 2024

Conversation

dscso
Copy link
Contributor

@dscso dscso commented Jan 22, 2024

I added support for universal binaries. The dmg contains an .app which runs native on aarch64 and x86 architecture. I tested it on my M1.
image

Copy link
Collaborator

@jacksongoode jacksongoode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

.github/workflows/build.yml Outdated Show resolved Hide resolved
@dscso
Copy link
Contributor Author

dscso commented Jan 22, 2024

If you want I can also change the name of the generated binary from Psst-x64.dmg -> Psst.dmg. I just was not sure if this would have side effects

@Insprill
Copy link
Collaborator

That would be good. Check to see if it's linked in the readme, I think that would be the only thing that needs to be updated.

Copy link
Collaborator

@jacksongoode jacksongoode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can merge, these linting issues are irrelevant to this PR.

@jacksongoode jacksongoode merged commit 38422b1 into jpochyla:master Jan 23, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants